Skip to content

Lower log level for missing plugin folders - #1060

Merged
HellAholic merged 1 commit into
5.14from
CURA-13388
Oct 20, 2026
Merged

HellAholic merged 1 commit into
5.14from
CURA-13388

Conversation

@HellAholic

Copy link
Copy Markdown
Contributor

Treat nonexistent potential plugin locations as informational rather than warning-worthy, and clarify the log message.

CURA-13388

Treat nonexistent potential plugin locations as informational rather than warning-worthy, and clarify the log message.
Copilot AI balanced review requested due to automatic review settings October 19, 2026 13:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The logging-only change matches the stated purpose and preserves existing control flow.

No open issues 🎉

What changed in this PR

Makes missing potential plugin folders informational rather than warning-worthy during plugin discovery.

Changes:

  • Changes the log level from warning to info.
  • Clarifies the message while preserving folder validation and early return behavior.
File Description
UM/​PluginRegistry.py Lowers the log level and revises the invalid-folder message.

Review effort: Balanced

@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Test Results

2 404 tests  ±0   2 389 ✅ ±0   18s ⏱️ -3s
    1 suites ±0      15 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit e696129. ± Comparison against base commit a080d47.

♻️ This comment has been updated with latest results.

@rburema R. Burema (rburema) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@HellAholic
HellAholic merged commit 537b5d0 into 5.14 Oct 20, 2026
17 checks passed
@HellAholic
HellAholic deleted the CURA-13388 branch October 20, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants